Skip to content

Remove unused arguments from zend_property_is_virtual()#21084

Merged
iluuu1994 merged 1 commit intophp:masterfrom
arshidkv12:zend_property_is_virtual
Jan 30, 2026
Merged

Remove unused arguments from zend_property_is_virtual()#21084
iluuu1994 merged 1 commit intophp:masterfrom
arshidkv12:zend_property_is_virtual

Conversation

@arshidkv12
Copy link
Copy Markdown
Contributor

@arshidkv12 arshidkv12 commented Jan 29, 2026 

- static bool zend_property_is_virtual(const zend_class_entry *ce, const zend_string *property_name, zend_ast *hooks_ast, uint32_t flags)
+ static bool zend_property_is_virtual(const zend_class_entry *ce, const zend_string *property_name, zend_ast *hooks_ast)

@TimWolla TimWolla requested a review from iluuu1994 January 29, 2026 22:57
@iluuu1994 iluuu1994 merged commit 3bff1dc into php:master Jan 30, 2026
10 checks passed
@iluuu1994
Copy link
Copy Markdown
Member

iluuu1994 commented Jan 30, 2026

Thanks @arshidkv12!

@arshidkv12
Copy link
Copy Markdown
Contributor Author

arshidkv12 commented Jan 30, 2026

You are welcome

@arshidkv12 arshidkv12 deleted the zend_property_is_virtual branch January 30, 2026 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iluuu1994 iluuu1994 Awaiting requested review from iluuu1994

Assignees

No one assigned

Labels

Category: Engine

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@arshidkv12 @iluuu1994